Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughAdds block-explorer links to network and mainnet docs, creates a new "Get WARD" guide, restructures the WARD airdrop doc into a "Withdraw WARD" flow with external-wallet instructions and exchange listings (including Korean doc), and a minor text fix in the Trade page. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/ward/get-ward.md`:
- Around line 5-12: The doc uses both "$WARD" and "WARD" interchangeably; update
the file to standardize on "WARD" (no dollar sign) throughout—replace all
occurrences of the token string "$WARD" in the heading and body (e.g., the
heading "# Get $WARD" and bullet lines mentioning "$WARD") with "WARD", ensure
the sentence "Please note that it's currently not possible to swap WARD in
Warden." remains consistent, and verify other mentions in this file match the
project's canonical "WARD" usage from mainnet-overview.md and ward-airdrop.md.
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@docs/help-center/docs/warden-app/ward-airdrop.md`:
- Around line 132-134: There's a typo in the admonition directive: locate the
malformed directive ':::not' used around the note block in ward-airdrop.md and
change it to the correct ':::note' so the note block renders properly; ensure
the opening and closing admonition markers match (':::note' ... ':::') and leave
the inner text unchanged.
- Around line 182-195: Remove or clarify the "Binance Alpha" list item: either
delete the "Binance Alpha" bullet (and its URL) or change its text to explicitly
state it is a Binance Wallet discovery/pre-listing page (not a Binance CEX spot
listing) and add a note that it is not an official Binance exchange spot market
unless an official Binance spot listing announcement for WARD is verified;
update the list entry referencing the "Binance Alpha" anchor text and URL
(0x6dc200b21894af4660b549b678ea8df22bf7cfac) accordingly.
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/developer-docs/docs/korean/ward.md (1)
1-1:⚠️ Potential issue | 🟡 MinorRemove the BOM before the front-matter delimiter.
The leading BOM character can break front‑matter parsing in some Markdown tooling.
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/ward/get-ward.md`:
- Around line 14-16: The "Centralized exchanges (spot)" list in get-ward.md
contains an Aster link whose URL path includes "futures", causing a spot vs
futures mismatch; update the Aster entry in the "Centralized exchanges (spot)"
section to either replace the link with Aster's spot trading URL or change the
label to "Centralized exchanges (futures)" so it matches the existing
https://www.asterdex.com/en/trade/pro/futures/WARDUSDT URL, making sure to edit
the Aster list item in docs/developer-docs/docs/ward/get-ward.md accordingly.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/developer-docs/docs/korean/ward.md`:
- Around line 141-150: Verify the Aster and Kraken entries: check the Aster link
and official exchange announcements and remove the "Aster" list item if it does
not officially support WARD (reference the "Aster" bullet), confirm the Kraken
endpoint actually provides a tradable WARD market and update or remove the
"Kraken" bullet accordingly, and annotate the "Binance Wallet (Alpha/Web3)"
entry to include/confirm the BEP-20 contract address
0x6dc200b21894af4660b549b678ea8df22bf7cfac or correct it if wrong.
🧹 Nitpick comments (1)
docs/developer-docs/docs/korean/ward.md (1)
136-142: Aster is a DEX, not a CEX — adjust the classification or sectioning.The “중앙화 거래소” list includes Aster, which presents itself as a decentralized perp DEX. Consider moving Aster under a separate “탈중앙화 거래소” section or renaming the heading to avoid misleading readers. (asterdex.io)
Suggested restructure
-**중앙화 거래소** -- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT) +**중앙화 거래소** +... + +**탈중앙화 거래소** +- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT)
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/help-center/docs/warden-app/ward-airdrop.md`:
- Around line 193-199: The entry "Aster" is misclassified under the
"**Centralized exchanges**" list and its URL "/trade/pro/futures/WARDUSDT"
points to a perpetuals/futures market; either move "Aster" out of the
"**Centralized exchanges**" section into a new "DEXs / Derivatives" or
"Decentralized exchanges" subsection and annotate that it offers perpetual
contracts, or replace the Aster link with the correct spot-market URL for WARD
(if available) and keep it only in the appropriate spot/DEX list; update the
link text and any note so readers aren’t directed to a leveraged derivatives
page when seeking to buy spot WARD.
- Line 171: The anchor in the sentence referencing the wallet main account is
incorrect: change the link fragment `#the-main-account` to `#main-account` so it
matches the heading `## Main account` used elsewhere (e.g., other references
that use `manage-your-wallets#main-account`); update the anchor in the line that
currently says `manage-your-wallets#the-main-account` to
`manage-your-wallets#main-account`.
| 1. [Add Warden as a custom network](https://docs.wardenprotocol.org/learn/warden-networks#add-warden-as-a-custom-network) to your external wallet. | ||
| 2. Copy your external EVM wallet address. | ||
| 3. Log in: 👉 [Warden](https://app.wardenprotocol.org) | ||
| 4. Click the wallet icon in the top-right corner to open your [main account](manage-your-wallets#the-main-account). |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# Find and read the relevant file sections
fd -e md -g '*ward-airdrop*' | head -1Repository: warden-protocol/wardenprotocol
Length of output: 122
🏁 Script executed:
# Check the file exists and read relevant lines
if [ -f "docs/help-center/docs/warden-app/ward-airdrop.md" ]; then
echo "=== Lines 27, 49, 74, 171 from ward-airdrop.md ==="
sed -n '27p;49p;74p;171p' "docs/help-center/docs/warden-app/ward-airdrop.md"
echo ""
echo "=== Lines 158-202 context ==="
sed -n '158,202p' "docs/help-center/docs/warden-app/ward-airdrop.md"
fiRepository: warden-protocol/wardenprotocol
Length of output: 3094
🏁 Script executed:
# Find manage-your-wallets.md and check headings
fd -e md -g '*manage-your-wallets*' | head -1 | xargs cat -n | grep -E "^[[:space:]]+[0-9]+[[:space:]]+(## |### )" | head -20Repository: warden-protocol/wardenprotocol
Length of output: 356
Fix broken anchor on line 171 — should be #main-account, not #the-main-account.
Lines 27, 49, and 74 all correctly reference manage-your-wallets#main-account, which matches the heading ## Main account in the manage-your-wallets.md file. Line 171 uses #the-main-account instead, causing the link to fail.
📝 Proposed fix
-4. Click the wallet icon in the top-right corner to open your [main account](manage-your-wallets#the-main-account).
+4. Click the wallet icon in the top-right corner to open your [main account](manage-your-wallets#main-account).📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| 4. Click the wallet icon in the top-right corner to open your [main account](manage-your-wallets#the-main-account). | |
| 4. Click the wallet icon in the top-right corner to open your [main account](manage-your-wallets#main-account). |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/help-center/docs/warden-app/ward-airdrop.md` at line 171, The anchor in
the sentence referencing the wallet main account is incorrect: change the link
fragment `#the-main-account` to `#main-account` so it matches the heading `##
Main account` used elsewhere (e.g., other references that use
`manage-your-wallets#main-account`); update the anchor in the line that
currently says `manage-your-wallets#the-main-account` to
`manage-your-wallets#main-account`.
| **Centralized exchanges** | ||
| - [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT) | ||
| - [Bitget](https://www.bitget.com/asia/spot/WARDUSDT) | ||
| - [Kraken](https://pro.kraken.com/app/trade/WARD-usd) | ||
| - [KuCoin](https://www.kucoin.com/trade/WARD-USDT) | ||
| - [LBank](https://www.lbank.com/trade/ward_usdt) | ||
| - [MEXC](https://www.mexc.com/exchange/WARD_USDT) |
There was a problem hiding this comment.
Aster is misclassified: it is a perpetuals DEX, not a centralized exchange, and the link goes to a futures market, not spot.
Two distinct accuracy problems here:
-
Wrong category: Aster is "a next-generation decentralized perpetual exchange." It is "a multi-chain DEX offering perpetual and spot trading." Listing it under Centralized exchanges is factually incorrect.
-
Wrong market type: The linked URL (
/trade/pro/futures/WARDUSDT) goes to the perpetuals/futures interface, not a spot market. Users who follow this link to "buy" WARD will land on a leveraged derivatives page, which is a materially different (and riskier) action than buying spot.
Either move Aster to a separate DEXs / Derivatives subsection with a note that it offers perpetual contracts, or replace the link with the Aster spot market URL if one exists for WARD.
📝 Suggested restructuring
**Centralized exchanges**
-
-- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT)
- [Bitget](https://www.bitget.com/asia/spot/WARDUSDT)
- [Kraken](https://pro.kraken.com/app/trade/WARD-usd)
- [KuCoin](https://www.kucoin.com/trade/WARD-USDT)
- [LBank](https://www.lbank.com/trade/ward_usdt)
- [MEXC](https://www.mexc.com/exchange/WARD_USDT)
+
+**Decentralized exchanges (derivatives)**
+- [Aster](https://www.asterdex.com/en/trade/pro/futures/WARDUSDT) (perpetual futures)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/help-center/docs/warden-app/ward-airdrop.md` around lines 193 - 199, The
entry "Aster" is misclassified under the "**Centralized exchanges**" list and
its URL "/trade/pro/futures/WARDUSDT" points to a perpetuals/futures market;
either move "Aster" out of the "**Centralized exchanges**" section into a new
"DEXs / Derivatives" or "Decentralized exchanges" subsection and annotate that
it offers perpetual contracts, or replace the Aster link with the correct
spot-market URL for WARD (if available) and keep it only in the appropriate
spot/DEX list; update the link text and any note so readers aren’t directed to a
leveraged derivatives page when seeking to buy spot WARD.
1 participant
Uh oh!
There was an error while loading. Please reload this page.